Skip to content

Conversation

@mddragnev
Copy link
Member

Closes #16187

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

ChronosSF
ChronosSF previously approved these changes Sep 29, 2025
@MayaKirova MayaKirova added 💥 status: in-test PRs currently being tested ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Oct 17, 2025
@kdinev kdinev requested a review from Copilot October 29, 2025 09:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a new README documentation for the Ignite UI Angular performance testing project. The documentation provides guidance on using the performance measurement infrastructure to analyze grid components.

  • New comprehensive README documenting the performance testing framework
  • Instructions for running performance tests and using the PerformanceService
  • Practical examples demonstrating how to measure grid sorting performance

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kdinev kdinev requested a review from Copilot October 30, 2025 08:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kdinev
Copy link
Member

kdinev commented Nov 20, 2025

@mddragnev cherry-pick this change to master, if you haven't already.

@kdinev kdinev merged commit 9af7c10 into 20.1.x Nov 20, 2025
5 checks passed
@kdinev kdinev deleted the mdragnev/performance-readme branch November 20, 2025 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

grid: performance version: 20.1.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants